fix(agents): align planner tools with spec §8.2 Plan Mode#181
fix(agents): align planner tools with spec §8.2 Plan Mode#181maystudios merged 1 commit intomainfrom
Conversation
…constraint Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Updates the planner agent template to better reflect Plan Mode expectations by adjusting its declared toolset and explicitly documenting planning-only constraints.
Changes:
- Updated
templates/agents/planner.mdfrontmatter tool list (removeBash; addEdit, web tools, todo tools, andAskUserQuestion) - Added a new
## Constraintssection describing Plan Mode write/edit limitations and the absence of Bash
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| ## Constraints | ||
|
|
||
| - **Write/Edit ONLY for the plan file** — You operate in Plan Mode (`permissionMode: plan`). Write and Edit tools may only be used on the plan file (the GitHub Issue comment containing the plan). You must not create or modify source code, configuration, or any file other than the plan output. |
There was a problem hiding this comment.
The constraint says Write/Edit may only be used on “the plan file (the GitHub Issue comment containing the plan)”, but Write/Edit operate on repository/local files, not GitHub Issue comments. Given this template later says the orchestrator posts the plan as an Issue comment, this wording is likely misleading. Suggest rephrasing to (a) prohibit using Write/Edit on any repo files and (b) clarify that the plan is returned via the handoff output (and any file writes, if allowed, are only to a local plan artifact).
| - **Write/Edit ONLY for the plan file** — You operate in Plan Mode (`permissionMode: plan`). Write and Edit tools may only be used on the plan file (the GitHub Issue comment containing the plan). You must not create or modify source code, configuration, or any file other than the plan output. | |
| - **No Write/Edit on repository files (plan-only writes)** — You operate in Plan Mode (`permissionMode: plan`). Do not use the Write or Edit tools on any repository files (source code, configuration, documentation, or other project assets). Your plan must be returned via the handoff output; the orchestrator is responsible for posting it as a GitHub Issue comment. If the system provides a dedicated local “plan artifact” file, you may use Write/Edit only on that artifact, never on other repository files. |
| ## Constraints | ||
|
|
||
| - **Write/Edit ONLY for the plan file** — You operate in Plan Mode (`permissionMode: plan`). Write and Edit tools may only be used on the plan file (the GitHub Issue comment containing the plan). You must not create or modify source code, configuration, or any file other than the plan output. | ||
| - **No Bash execution** — You do not have the Bash tool. All investigation is done via Read, Grep, Glob, WebSearch, and WebFetch. |
There was a problem hiding this comment.
This “No Bash execution” constraint may conflict with the later task format requirement that each task include a <verify> command “runnable via Bash”. Consider clarifying here that the planner must specify Bash-runnable verification commands for the executor to run, but must not execute Bash themselves.
| - **No Bash execution** — You do not have the Bash tool. All investigation is done via Read, Grep, Glob, WebSearch, and WebFetch. | |
| - **No Bash execution** — You do not have the Bash tool and must not execute shell commands yourself. All investigation is done via Read, Grep, Glob, WebSearch, and WebFetch. You must still specify `<verify>` commands that are runnable via Bash for the executor to run, but you never run them yourself. |
|
🎉 This PR is included in version 5.13.2 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Summary
templates/agents/planner.mdtools list to match spec §8.2 Plan Mode: removedBash, addedEdit,WebSearch,WebFetch,TodoRead,TodoWrite,AskUserQuestion## Constraintssection enforcing write/edit restriction to plan file only and documenting the absence of BashTest plan
🤖 Generated with Claude Code